treeview: Don't underallocate editing widgets that are too big

combobox: Unconditionally draw background and border

combobox: Remove autoscroll timer on popdown

combobox: Remove cell_view_frame

Cell views can draw their own frames now.

cellview: Implement border and background drawing

gtkimagemenuitem: improve the replacement docs even more

Since the stock strings usually have menmosics suggest
gtk_menu_item_new_with_mnemonic().

offscreen_window: Remove deprecated API calls

gtkimagemenuitem: fix suggestion for replacement

gtk_menu_item_new(9 would create an empty menu item. While the stock deprecation
aims to unclutter the menus, I feel this goes too far.

drawingarea: use same background approach as GtkLayout

Call gtk_style_context_set_background() every time the style is updated
and add a comment as to why we still need to do it here.

https://bugzilla.gnome.org/show_bug.cgi?id=752599

Revert "drawingarea: don't call gtk_style_context_set_background()"

This reverts commit 6786c9d724ad57ea30e169d2afcb4a314b2e56bb.

https://bugzilla.gnome.org/show_bug.cgi?id=752599

Revert "drawingarea: checks parent's draw() before call it"

This reverts commit b3166b745f41a901171821ccc9f0c69b9cc35e70.

https://bugzilla.gnome.org/show_bug.cgi?id=752599

file chooser: Memory handling fixes

I forgot to clean up the long press gesture, and in additon, creating
the rename popover from the ui template was causing the tree view
to not be disposed when the file chooser goes away. Work around this
by manually unsetting the relative-to widget of the popover in dispose.

Fix return value error in gtk_stack_get_interpolate_size

Updated Spanish translation

Updated Spanish translation

wayland: Always apply scaling factor to cursor hotspot and dimension

Prior to this patch, the hotspot would be passed in buffer coordinate
space. Where this were ever tested, i.e. in a patched mutter, the
server interpreted them incorrectly, which meant it went undiscovered.
In the updated mutter patches the incorrect behavior in GTK+ was
discovered due to the behavior in mutter was corrected.

In the themed cursor case, the dimensions were not correctly scaled
either, but this had no negative visible effect because the dimension is
only used for reporting damage tracking, and passing a bigger damage
region than surface has no negative visible effects.

https://bugzilla.gnome.org/show_bug.cgi?id=752616

GtkPopover: Set interpolate-size on the stack...

... in gtk_popover_bind_model
and gtk_popover_menu_init

GtkStack: Add interpolate-size property

And use it to determine whether we should actually interpolate between
stack sizes or not.

GtkListBox: Improve focus scrolling

We are trying to scroll the header in view together with the
focus row. The way this is implemented works fine when scrolling
up, but falls short when scrolling down. Fix this by making sure
that both the row and the header bar visible.

GtkListBox: Listen for adjustment changes

We automatically pick up an adjustment from our parent
scrollable, but we failed to update it when it changes.
This is happening in the places sidebar, and it was causing
the focus-tracking to fail there, letting the focus move
out of view. With this change, the focus remains visible.

places sidebar: Improve keynav

Remove the code in the sidebar keypress handler that imperfectly
reproduces what GtkListBox already does: moving selection and
focus in response to arrow keys.

trivial: Some reformatting

places sidebar: Remove pointless code

No need to set or unset adjustments explicitly.

spin button: Add Home and End as key bindings

Make these go to the lower/upper boundary of the adjustment, like
Ctrl-PageUp/PageDown already do.
https://bugzilla.gnome.org/show_bug.cgi?id=309300

GtkWidget: Add custom font map support

This allows to use a custom font map for parts of the widget
hierarchy - this can be used for application-specific fonts.

https://bugzilla.gnome.org/show_bug.cgi?id=751812

font chooser: Add placeholder text to preview entry

This will make things more understandable if an application
sets an empty preview text.
https://bugzilla.gnome.org/show_bug.cgi?id=725551

GdkGLContext little fix to property names and descriptions.

https://bugzilla.gnome.org/show_bug.cgi?id=752502

Updates for 3.17.5

builder-tool: Don't strip border-width

Strictly speaking, this would only have to be special-cased inside
GtkDialog, but lets just skip it altogether.

Fix csd decorations

By accident, commit b9b17c3c2a434983452561c3 removed one line
that wasn't unused, but important for hiding close buttons on
dialogs. Bring it back.

Updated Hebrew translation

window: Set the type hint when it changes, not just on map()

There is no reason to wait until map() to reset a type hint on the GDK
window. Just set it when it changes.

window: g_object_notify() on type-hint change

window: Don't update type hint when it didn't change

window: Allow setting type hint on mapped windows

window: Don't keep 2 type hints

1. This confuses the code as it's using the old type hint with the new
type hint on GDK window creation

2. It only existed as a workaround for old code that directly accessed
window->type_hint which hasn't been possible since 3.0.

Updated Portuguese translation

Updated Portuguese translation

Updated Czech translation

combobox: Simplify code

We don't add new children to the child's parent, we add them to ourself.
Because guess who a child's parent is?

combobox: Remove unused header function functionality

combobox: Remove unused argument from private function

Fix the build

Revert "GtkGrid: Add a few NULL checks"

This reverts commit e215db6da21fc7c54f49b247afabc06a18d60513.

We are actually requiring non-NULL out arguments here now.

testgtk: Remove pointless code

The display is getting overwritten immediately. Pointed out by
coverity.

inspector: Avoid a NULL deref

Coverity pointed out that getting a class name requires
dereferencing the pointer, so only don't do it if it is NULL.

GtkWindow: Fix a use-after-free

We were printing a string in a warning after freeing it.
Also, fix an x/y typo that might cause popovers to be
mispositioned.

file chooser: remove an unused GError variable

Pointed out by coverity

inspector: Remove some dead code

Not sure what I was thinking here. Coverity pointed this out.

inspector: Avoid a NULL deref

treeview: Add an explicit return for clarity

I believe that first_column can only ever be NULL here if
last_column is NULL too, in which case we'd exited already.
But coverity doesn't see that, so add an explicit exit.

tooltip: Remove unnecessary checks

Coverity uses these as clues to complain.

list box: Remove unnecessary checks

Again, coverity takes these as clues to complain.

GtkLabel: Remove an unnecessary check

Coverity has the annoying habit of guessing backwards that
if we check the value here, it might be NULL earlier.

GtkGrid: Add a few NULL checks

Coverity complained about these.

GtkActionHelper: Avoid null deref in debug code

gdk: Add some parameter checking

This may help coverity recognize that we're not called with
invalid combinations.

testdialog: Error out if templates are missing

Coverity complained about us ignoring the return value
here, and it is nicer to get an error message about this.

cups: Annotate a call whose return value we don't care about

We're closing this fd, but we're not going to do anything else
if that fails. Annotate to let coverity see this.

inspector: Check a return value

We don't care about the error here, but we were relying on
g_resources_get_info to zero the size even in case of error.
No need to do that, we can just check the return value. Plus,
it makes coverity happy.

inspector: Annotate a call whose return value we don't need

The way this code is written, we know that there is an item in
the hash table and the iter_next call will give it to us, so
no need to check the return value. Annotate the call to tell
coverity.

gtkicontheme: Check a return value

We were already looking at the error anyway, but rewriting things
this way lets coverity see the light.

gtkbuilderparser: Add some assertions

Add some assertions that things are non-NULL when we know they
are, so that coverity gets it.

gtk-demo: Check a return value

Found by coverity.

css: Fix GtkCssChange enum

Previously, the (1<<31) value evaluated to the 64bit value
  0xFFFFFFFF80000000
instead of
  0x0000000080000000
Avoid this by explicitly casting the value to unsigned long long.

Also cast all values to unsigned long long to achieve consistency
and hopefully in the future get new values added the same way.

headerbar: Switch css positions on direction change

Updated Hungarian translation

file chooser: Improve popover positioning

We keep the popover vertically tied to the selected row, but pick
up the horizontal position from the click / touch.

places sidebar: Preserve ::populate-popup functionality

We "fix" this signal the same way we did for GtkEntry and
GtkTextView: slightly change the signature so that the passed
widget may be a container other than a GtkMenu, and add a
property to explicitly opt in to receiving the signal in
this case.

file chooser: Use a popover for the context menu

Use a popover for the context menu, and add a long press gesture
to allow opening it with touch.

places sidebar: Use a popover for the context menu

Use a popover for the context menu, and add a long press gesture
to allow opening it with touch.

gdk: fix crash if visual is NULL

cairo_xlib_surface_get_visual might return NULL so we need to
check for NULL before trying to use it.

https://bugzilla.gnome.org/show_bug.cgi?id=752520

inspector: Avoid a gratitious use of g_strv_length

No need to count the strings first before iterating over them.

file chooser: Make bindings work in save entry

With the name entry being in the header bar now, we no longer
get bindings working by just letting the key event bubble up,
we have to explicitly apply them on key events that the save
entry is not handling.

GtkFileChooserButton: Don't update icon/label so often

They were updated in style-changed, causing the label to get set to
(None), then to the actual file name again a frame later, both of the
updates cause the GtkFileChooserButton to resize, possibly to the
minimal width, causing the layout to jump. Fix this by only updating
icon/label in style-updated when the icon theme actually changed, which
is the only case we care about here.

https://bugzilla.gnome.org/show_bug.cgi?id=752509

gtkprintunixdialog: Set border_width to 0

Avoid O(n²) walking of string arrays

"Yo, we heard you like traversing NULL-terminated arrays to operate on
them, so we called g_strv_length() as the for condition, so you can
iterate the array while iterating the array."

Instead of making famed rapper and television producer Xzibit proud, we
should avoid calling g_strv_length() on an array while looping on the
array, to avoid quadratic complexity.

We do this in various places that deal with arrays of strings that we
cannot really guess are short enough not to matter — e.g. the list of
CSS selectors in the inspector, or the required authentication
information for printing.

placessidebar: show persistent drives when not showing Other Locations

Previous commits changes the behavior of GtkPlacesSidebar, as it stops
handling persistent devices and only manages mounted networks, XDG
directories and bookmarks.

By radically breaking the previous behavior, we may have trouble
since we provide no alternatives to that besides the private widget
GtkPlacesView.

Fix that by showing the persistent devices when not showing Other
Locations item.

tests/gears: Ensure we don't use unavailable GL resources

We need to check on realize if we have access to a GL context, before
calling GL functions. We use gtk_gl_area_get_error() for that.

We also need to tear down the resources during unrealization, instead
of leaking them.

placessidebar: fix Connect to Server position

Commit 7db399d975b8f9626c21761dde5f2d5feeb6e305 introduces
the Other Locations item, as well as some behavioral changes
to GtkPlacesSidebar. It, howevers, wrongly changes the section
of code that handles the Connect to Server item, with various
side effects depending on the environment setup.

Fix that by adding the Connect to Server at the right section
of code.

Updated Hebrew po-properties translation

Updated Hebrew translation

Updated Hebrew translation

Updated Spanish translation

Updated Spanish translation

Updated POTFILES.in and POTFILES.skip

tests: Remove testplacesview test

The GtkPlacesView widget was made private, so we can't create it in a
testcase.

tests: Fix gcc complaints about testfilechooser

headerbar: Get rid of get_path_for_child()

Instead, reorder CSS nodes.

headerbar: Finish widget setup before showing it.

headerbar: forall() the button boxes in the right order

headerbar: Remove unused variables in private struct

toolbar: Switch from get_path_for_child() to css node managing

cssnode: Add gtk_css_node_reverse_children()

cssnode: Change API of some functions

gtk_css_node_set_after/before() are now called
gtk_css_node_insert_after/before().

This brings them in line with other similar APIs (ie GtkListStore). And
it allows easier usage of the API (see changes to gtkbox.c).

filechooserwidget: use places view to manage fixed devices

Previous patch modified places sidebar widget to stop handling
fixed devices by adding an "Other Locations..." item. Up to now,
however, these changes are isolated from each other since the
bundled file manager widgets ignore the sidebar requests for
external management of fixed devices and networks.

To fix that, make the file chooser widget be aware of the
GtkPlacesSidebar::show-other-locations signal and, when requested,
show places view to manage the fixed devices and networks.

https://bugzilla.gnome.org/show_bug.cgi?id=752034

placessidebar: add Other Locations item

Places sidebar is a widget that enabled the user to select
XDG directories, bookmarks and mounted network locations,
as well as manages permanent and removable devices.

The new design that aims to look less clutered makes the
sidebar display only removable devices, as well as mounted
networks, bookmarks and XDG directories, and delegates the
management of permanent devices such as hard drive partitions
to GtkPlacesView, a newly introduced widget for this specific
purpose.

To delegate it, add an "Other Locations..." item to notify
when the permanent devices manager is required. Besides that,
don't show these fixes devices on the sidebar itself, as they
are not supposed to be handled by the sidebar anymore.

https://bugzilla.gnome.org/show_bug.cgi?id=752034

placesview: add view for fixed drives and networks

Places sidebar shows XDG directories, mounted and unmounted devices,
connected networks, bookmarks and actions like 'Connect to server'
and 'Insert location', which causes the sidebar to grow very quickly
and look cluttered. Because of that, new mockups for the sidebar try
to simplify it.

To make the sidebar simpler, the new mockups propose that it should
only handle connected networks and removable devices such as flash
drives and USB devices, and delegates other devices for external
widgets through the 'Other Locations' item.

To handle fixed devices and manage network connections, add a new
widget named GtkPlacesView, based on Nautilus mockups to keep
consistency between GNOME file management tools - in this case,
between Nautilus and the bundled Gtk's file chooser.

https://bugzilla.gnome.org/show_bug.cgi?id=752034

Revert "x11: Query pointer devices' scroll valuators on toplevel enter events"

This reverts commit 77b8495bc4e51826afa451914d6a3b0df1b62a24.

The commit broke more scenarios than fixed, better to go back to square one.

GdkDisplayX11: Properly translate server timestamps from _NET_WM_FRAME_* messages

When using frame times from _NET_WM_FRAME_DRAWN and _NET_WM_FRAME_TIMINGS, we
were treating them as local monotonic times, but they are actually extended-precision
versions of the server time, and need to be translated to monotonic times in the
case where the X server and client aren't running on the same system.

This fixes rendering stalls when using X over a remote ssh connection.
https://bugzilla.gnome.org/show_bug.cgi?id=741800